Skip to content

Conversation

finestructure
Copy link
Member

@finestructure finestructure commented Jan 1, 2025

This moves fetchMetadata to GithubClient.

@cla-bot cla-bot bot added the cla-signed label Jan 1, 2025
name: "SPI-Server",
platforms: [
.macOS(.v13)
.macOS(.v15)
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

macOS 15 is required for typed throw runtime support due to our use of

        @Dependency(\.github.fetchMetadata) var fetchMetadata

below.

}
}
}
}
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This test is now superfluous. It tested the parseOwnerName part of the static func fetchMetadata(client: Client, packageUrl: String) async throws → Metadata overload that was actually not used and which I removed.

@finestructure finestructure marked this pull request as ready for review January 1, 2025 17:03
@finestructure finestructure force-pushed the issue-3469-dependency-transition-14 branch from 85201b5 to d1f9586 Compare January 3, 2025 09:34
@finestructure finestructure merged commit 8797eb4 into main Jan 3, 2025
6 checks passed
@finestructure finestructure deleted the issue-3469-dependency-transition-14 branch January 3, 2025 09:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants